Note

Medium Risk
Changes error types and messages across all VCS action entry points and progress consumption; UI that matched on old string messages may need updates, but behavior for successful paths is unchanged.

Overview
Replaces ad-hoc string failures for client source-control actions with Effect Schema tagged errors that carry operation, target, and action identity instead of copying sensitive remote text into error messages.

VcsActionUnavailableError now includes operation, environmentId, and cwd with a derived user-facing message; web hooks in sourceControlActions.ts build this inline after removing unavailableResult. consumeVcsActionProgress splits execution failures into VcsActionRemoteFailureError (phase + remoteMessageLength only) and VcsActionMissingTerminalEventError when the progress stream never terminates; VcsActionExecutionError is now a union of those two. parseVcsActionTargetKey is schema-validated and throws VcsActionTargetKeyParseError with key length and native cause, not the raw key. The stacked-action command cache keys unavailable targets per partial environmentId/cwd rather than a single shared unavailable key.

Tests assert diagnostics stay structural and do not embed credentials or full remote payloads in .message.

^{Reviewed by Cursor Bugbot for commit fa41678. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Structure VCS action errors with operation, environmentId, and cwd context

• Replaces free-form message strings in VcsActionUnavailableError with structured fields (operation, environmentId, cwd) and a standardized message getter across all VCS action hooks in sourceControlActions.ts.
• Introduces two new error classes in vcsAction.ts: VcsActionRemoteFailureError (for action_failed events, omitting remote message body but recording its length) and VcsActionMissingTerminalEventError (for protocol cases with no terminal event).
• Adds VcsActionTargetKeyParseError for invalid target key parsing, reporting only key length to avoid leaking sensitive content.
• Removes the unavailableResult helper and replaces all call sites with direct AsyncResult.failure construction using the new structured error shape.
• Behavioral Change: callers of consumeVcsActionProgress and track() now receive specific typed error variants instead of a generic error; remote failure messages are no longer forwarded to the client.

^{Macroscope summarized fa41678.}